perf(render): encode primitive fingerprints without strings#63
perf(render): encode primitive fingerprints without strings#63TypeThe0ry wants to merge 1 commit into
Conversation
b330197 to
3c4230c
Compare
Trusted performance decisionGitHub A/B run Functional Build/Test are green and the trusted infrastructure completed normally. The measured result did not clear the optimization threshold, so this candidate will not be merged or described as an optimization. Decision artifact: |
Final paired A/B result: rejectedFormal decision:
The A/A calibration also detected drift in the precompute sub-profile, but the tiles regression is large and directionally consistent on its own. Closing without merge. |
What changed
boolean,char,int, andlong:separators and exactObjects.toString(value, "")character contract, including zero and all MIN/MAX boundariesWhy
A full table render computes hundreds of region/tile fingerprints. Primitive fields currently box and materialize temporary decimal strings before each per-character FNV pass. The new encoder packs reverse decimal digits into two local longs, then emits them in the original order; it handles
Long.MIN_VALUEin the negative domain to avoid overflow.This deliberately does not use the behavior-changing type-marker/binary encoding from the dirty development tree.
Validation
No Gradle, JMH, test, or server workload was run locally. GitHub Build/Test is the functional gate. This PR remains draft and must not merge until #60 and the base-owned region profile in #62 land, then #62's independent exact-FNV oracle and time/allocation A/B must report
PASS_OPTIMIZED.